Auto merge of #1714 - alexcrichton:default-feature-oscillating, r=huonw
authorbors <bors@rust-lang.org>
Sun, 14 Jun 2015 20:24:01 +0000 (20:24 +0000)
committerbors <bors@rust-lang.org>
Sun, 14 Jun 2015 20:24:01 +0000 (20:24 +0000)
commit47735b16eb925935e266160a6529f0fc3432ef63
tree5e6cad141224e96fd18caf69c7e748914c405c07
parentba47db22c28e4e7213d79c64f91141dd3a71ccb4
parent0ba5a7a2c3d5dd1adff874adbc2c9be3198ad1f9
Auto merge of #1714 - alexcrichton:default-feature-oscillating, r=huonw

There was previously a bug in resolve where turning off the default set of
features would cause resolve to not correctly settle on the set of features
activated for a package. If one dependency turned off all features, and then the
next dependency to be activated only turned on the default feature, the default
feature wouldn't actually end up getting activated.

This commit alters the check to see if the package has been previously activated
to more rigorously check to see if the 'default' feature is activated
previously. If a dependency doesn't use the default feature, or if the package
itself does not have the feature "default", then the package is considered
activated, but otherwise it needs to go through another cycle of resolution.